Skip to content

feat: add plan table course search and add#93

Merged
chenxin-yan merged 16 commits intomainfrom
feat/plan-add-course-87
Dec 10, 2025
Merged

feat: add plan table course search and add#93
chenxin-yan merged 16 commits intomainfrom
feat/plan-add-course-87

Conversation

@xyspg
Copy link
Copy Markdown
Collaborator

@xyspg xyspg commented Nov 10, 2025

📌 What's Changed
add plan table course search and add. Closes #87

drag and drop support.

CleanShot.2025-11-10.at.02.36.06.mp4

✅ Actions

Known issue:

📝 Notes for Reviewer

@xyspg xyspg marked this pull request as ready for review November 10, 2025 16:34
@xyspg xyspg requested a review from chenxin-yan November 10, 2025 16:34
@xyspg xyspg marked this pull request as draft November 10, 2025 16:47
@chenxin-yan
Copy link
Copy Markdown
Member

chenxin-yan commented Nov 11, 2025

regarding #92 , FYI, the First year and graduation year is determined through the date in students record not through parsing. I think that's where the inconsistency come from as the seeding data for student starts at 2023 fall. Good call tho. I will change it so that instead of letting user fill that out during onboarding, it will be inferred from Degree Progress Report. User can later edit the starting and graduation date in user profile page (#83).

@chenxin-yan chenxin-yan force-pushed the main branch 4 times, most recently from e2d6c6a to f1d9b93 Compare November 17, 2025 23:10
@xyspg xyspg marked this pull request as ready for review December 8, 2025 01:42
@xyspg xyspg marked this pull request as draft December 8, 2025 01:45
@xyspg xyspg marked this pull request as ready for review December 8, 2025 03:52
@xyspg
Copy link
Copy Markdown
Collaborator Author

xyspg commented Dec 8, 2025

I have finished this PR. But one thing is that some course, for example, CSCI-UA 102 Data Structures only offer in Fall / Spring. but there is no such term info in our db. we should throw an error if user tries to add this course to summer.

@chenxin-yan chenxin-yan merged commit e6fb64c into main Dec 10, 2025
2 checks passed
@chenxin-yan chenxin-yan deleted the feat/plan-add-course-87 branch December 10, 2025 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add plan table course search and add

2 participants